Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added stdin reading capability to amqpsend #4

Merged
merged 1 commit into from Mar 2, 2012
Merged

Added stdin reading capability to amqpsend #4

merged 1 commit into from Mar 2, 2012

Conversation

astephens25
Copy link
Contributor

I added the option to leave out the filename and the command line message and instead read the message body from standard input.

I performed regression testing to make sure the previous options still behaved generally as expected. The primary difference is where the help message was printed if the filename option and command-line message argument were missing.

Additional testing included sending of large files (20M) to test buffer reallocations. I did not do any performance analysis using the read buffer of only 8K.

Feel free to not accept these changes as you see fit. I made them for my own use and wish to give you the option to make them generally available in your project if you deem it desirable to do so.

Thank you.

rmt added a commit that referenced this pull request Mar 2, 2012
Added stdin reading capability to amqpsend
@rmt rmt merged commit 29041cc into rmt:master Mar 2, 2012
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants